added help comment in setupForm example #38
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
to help newbies who try to put all scripts in DOM head
|
Great! |
victorjonsson
added a commit
that referenced
this pull request
Jul 20, 2013
added help comment in setupForm example
|
Can you put your website docs on GitHub so they can be edited as new code is merged? ... sent from jphone On Jul 20, 2013, at 2:51 AM, Victor Jonsson notifications@github.com wrote:
|
|
That's a great idea! |
|
thanks! i am working on a custom validator to handle groups of checkboxes, to make sure the qty checked meets validation rules, ie. min 1 or max 5 etc... almost done, but probably won't get it ready to pull until Monday. i abstracted the range, min & max qty tests to a separate function so it can be reused by other validators. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
to help newbies who try to put all scripts in DOM head